-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
create keycloak #2
base: main
Are you sure you want to change the base?
Conversation
keycloak/main.tf
Outdated
chart = "keyclok" | ||
chart_version = var.chart_version | ||
conf = merge(local.conf_defaults, var.conf) | ||
conf_defaults = merge({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about another param? Like an admin password or ingress? Just install helm chart not enough for us
fix keycloak
@jamal68 pls squash git history. Too many similar commit messages. |
|
||
## Example how add with module | ||
``` | ||
module "keycloak" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you show full example with google auth values and depend-on?
locals { | ||
name = "kube-keycloak" | ||
repository = "https://github.com/bitnami/charts/tree/master/bitnami/keycloak/" | ||
chart = "kube-keycloak" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
chart name is keycloak (without kube- prefix)
|
||
locals { | ||
name = "kube-keycloak" | ||
repository = "https://github.com/bitnami/charts/tree/master/bitnami/keycloak/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
repository incorrect, this is Github, needs helm chart repo https://charts.bitnami.com/bitnami
conf = merge(local.conf_defaults, var.conf) | ||
password = var.keycloak_password == "" ? random_password.keycloak_password.result : var.keycloak_password | ||
conf_defaults = { | ||
"keycloak.enabled" = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
incorrect values from 144-154, check https://github.com/bitnami/charts/blob/master/bitnami/keycloak/values.yaml
description = "A name of namespace for creating" | ||
} | ||
|
||
variable "module_depends_on" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
after Terraform update 0.14 this variable deprecated - use depends_on in module level
No description provided.